Skip to content

Conversation

@nexero
Copy link

@nexero nexero commented Jul 17, 2016

Had to change some files for successful building with linux platform. not sure, if it breaks something for win32 or macos, though..

@antrad
Copy link
Owner

antrad commented Jul 21, 2016

Hello. I browsed through your changes and I guess most of it should be fine (though I have very limited/no knowledge of Cmake and cross platform programming), but I currently don't have the PC set up to test it, I'm having a break from coding and all.

@tsdko
Copy link

tsdko commented Aug 25, 2016

Had to change a bit more to get it to compile on Gentoo:

diff --git a/src/common.h b/src/common.h
index ee45256..a798dfb 100644
--- a/src/common.h
+++ b/src/common.h
@@ -16,6 +16,7 @@
 //
 #include <SDL2/SDL_config.h>//AR (#include <SDL_config.h>)
 #include <stdio.h>
+#include <stdint.h>

 #ifdef _MSC_VER
 // For simplicity sake, just make snprintf sprintf_s even though they aren't quite the same
diff --git a/src/sdlport/video.cpp b/src/sdlport/video.cpp
index b126044..9a0247a 100644
--- a/src/sdlport/video.cpp
+++ b/src/sdlport/video.cpp
@@ -27,7 +27,7 @@
 #ifdef WIN32
 #include <windows.h>
 #endif
-#include "GLee.h"
+#include <GL/GLee.h>
 //Abuse linker->input:
 // - opengl32.lib
 // - GLee.lib
diff --git a/src/tool/AR_SPEC.h b/src/tool/AR_SPEC.h
index 70e1d60..edc1359 100644
--- a/src/tool/AR_SPEC.h
+++ b/src/tool/AR_SPEC.h
@@ -25,7 +25,6 @@
 // - linker
 //     cv210.lib
 //     cxcore210.lib
-//     highgui210.lib
 // - libraries
 //     OpenCV2.1\lib
 // - include
@@ -60,7 +59,6 @@
 #include <sstream>

 #include <opencv2/opencv.hpp>
-#include <opencv2/highgui.hpp>
 using namespace cv;

 #include "common.h"

@monreal
Copy link

monreal commented Sep 11, 2017

I just compiled this on Fedora 26. Needed to apply both patchsets.

@tuomosipola
Copy link

tuomosipola commented Jun 7, 2019

Just compiled on Ubuntu 19.04. Applied both patches.

In addition, GLee seems to be unmaintaided (and not included in Ubuntu), so the more modern glew should be used in src/sdlport/video.cpp: change #include "GLee.h" to #include <GL/glew.h>

@ghost
Copy link

ghost commented Oct 24, 2019

@tuomosipola Could you try my fork? It uses Glad instead of GLee or glew, as well as some other changes.

@tuomosipola
Copy link

@fluffrabbit Sorry, I just noticed this message. Tried it. It compiles and I could play a bit. No sound, but I don't think even this fork had them.

@ghost
Copy link

ghost commented Mar 2, 2020 via email

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants